-
Notifications
You must be signed in to change notification settings - Fork 357
Add Forgot Password link on Sign In page #1370
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Add Forgot Password link on Sign In page #1370
Conversation
|
@pratyushjha06 is attempting to deploy a commit to the avdheshvarshney's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
Thank you for submitting your pull request! We'll review it as soon as possible. For further communication, join our discord server https://discord.gg/tSqtvHUJzE. |
Avdhesh-Varshney
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@pratyushjha06 use nodemailer to send mail. Create api for this
| Enter your email address and we'll send you a link to reset your | ||
| password. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Update the text by Enter your email address to receive a link to reset your password.
| e.preventDefault(); | ||
| setLoading(true); | ||
|
|
||
| const formData = new FormData(e.currentTarget); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Avoid form data, try to use state management of email
| const formData = new FormData(e.currentTarget); | ||
| const email = formData.get('email'); | ||
|
|
||
| // TODO: Implement password reset API call here |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Implement api call
| import { LoginLazyComponent } from '../components'; | ||
|
|
||
| // Add lazy loading for ResetPassword component | ||
| const ResetPasswordLazyComponent = lazy( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Put it in components/index.tsx and import from there similar to login page
|
@pratyushjha06 Please complete all mentioned changes |
Pull Requests Review Criteria
Caution
PRs that fail to meet these review standards will be automatically flagged and may be rejected by maintainers.
mainCloses: #1277
Describe the add-ons or changes you've made 📃
Added a "Forgot Password?" link on the Sign In page that redirects users to a password reset page.
Changes:
/reset-passwordrouteThe UI is complete. Backend email functionality will be added separately.
Screenshots 📷
09-22-14.-.Trim.mp4
Note to reviewers (Optional) 📄